Skip to content

fix: add cronJob tool access to the orchestrator - #875

Merged
avoidwork merged 3 commits into
mainfrom
fix/add-orchestrator-cronjob-access
Aug 25, 2026
Merged

fix: add cronJob tool access to the orchestrator#875
avoidwork merged 3 commits into
mainfrom
fix/add-orchestrator-cronjob-access

Conversation

@avoidwork

Copy link
Copy Markdown
Owner

Description

Add cronJob tool access to the orchestrator agent. The orchestrator previously lacked the ability to manage scheduled cron jobs during interactive sessions. This fix adds cronJob to the ORCHESTRATOR_TOOLS array and classifies it for the orchestrator agent type.

Type of Change

  • Bugfix (non-breaking change which fixes an issue)

Testing

  • Created tests/unit/tools_orchestrator.test.js with 7 tests verifying:
    • cronJob is included in ORCHESTRATOR_TOOLS array
    • ORCHESTRATOR_TOOLS is an array of strings
    • cronJob is classified for orchestrator agent type
    • cronJob retains existing classifications (security-audit, performance)
    • getToolsForAgentTypes returns cronJob for orchestrator type
    • getToolsForAgentTypes returns cronJob for security-audit type
    • getToolsForAgentTypes returns cronJob for performance type
  • Full test suite: 1288 tests pass (0 failures)
  • `npm run lint}: clean (0 warnings, 0 errors)
  • `npm run coverage}: coverage report generated successfully

Coverage

  • Line coverage maintained

Checklist

  • npm run lint passes
  • Tests pass with maintained line coverage
  • No forbidden patterns used
  • Conventional Commit style applied

Closes #874

Add OpenSpec change for issue #874: add cronJob tool access to the
orchestrator. Includes proposal.md, tasks.md, and .openspec.yaml.
- Add "cronJob" to ORCHESTRATOR_TOOLS array in src/tools/index.js
- Add "orchestrator" to cronJob classification in TOOL_CLASSIFICATIONS
- Update README.md Built-in Tools table to document orchestrator access
- Add unit tests verifying orchestrator has access to cronJob (7 tests)

Closes #874
Archived change to openspec/changes/archive/2026-08-25-add-orchestrator-cronjob-access/.
@avoidwork

Copy link
Copy Markdown
Owner Author

Audit Results — Issue #874: Add cronJob Tool Access to the Orchestrator

Summary

Fix implemented and verified. The orchestrator agent now has access to the cronJob tool.

Changes Made

File Change
src/tools/index.js Added "cronJob" to ORCHESTRATOR_TOOLS array (alphabetically ordered)
src/tools/index.js Added "orchestrator" to cronJob classification in TOOL_CLASSIFICATIONS (alphabetically ordered)
README.md Updated cronJob row in Built-in Tools table to note orchestrator access
tests/unit/tools_orchestrator.test.js New test file with 7 tests

Test Results

  • Unit tests: 1288/1288 pass (0 failures)
  • New tests: 7/7 pass (tools_orchestrator.test.js)
  • Lint: oxlint — 0 warnings, 0 errors
  • Format: oxfmt — all files pass
  • Coverage: npm run coverage — report generated successfully

App Verification

  • timeout 8 node index.js --mode chat test — app starts and responds correctly

OpenSpec Change

  • Change add-orchestrator-cronjob-access archived to openspec/changes/archive/2026-08-25-add-orchestrator-cronjob-access/
  • All tasks marked complete in tasks.md

Risk Assessment

  • Low risk: Only two array modifications in src/tools/index.js, one README update, and new tests
  • No new dependencies
  • No breaking changescronJob tool already exists and is fully functional
  • Backward compatible — existing agent types retain their classifications

@avoidwork
avoidwork merged commit fd3ba62 into main Aug 25, 2026
2 checks passed
@avoidwork
avoidwork deleted the fix/add-orchestrator-cronjob-access branch August 25, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: add cronJob tool access to the orchestrator

1 participant